-
Notifications
You must be signed in to change notification settings - Fork 14
Added mobile & accessible changes #68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
e36ba18
to
4b54e32
Compare
src/styles/_variables.scss
Outdated
|
||
// Sizes | ||
$hacks-page-width: 981px; | ||
$hacks-page-width: em(981); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fancy sass 😮
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
undid it because it don't work doe
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you baited me into reading https://adamwathan.me/dont-use-em-for-media-queries/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wait where's the undo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to scream brb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
desktop broken (content is left aligned instead of centered)
fixed desktop version -- it was because of container widths |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved because we're agile (i.e. too jaded to pick it apart)
mobile still sporting the boomer's iPhone font size .-.
maybe you're going tor the picture book aesthetic |
I don't know what aesthetic I'm going for please |
- Smaller font size and final changes
647ab90
to
24abcf5
Compare
Fixes #65